Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore: enhance ci reports #497

Merged
merged 14 commits into from
May 6, 2024
Merged

chore: enhance ci reports #497

merged 14 commits into from
May 6, 2024

Conversation

nicholasgriffintn
Copy link
Member

@nicholasgriffintn nicholasgriffintn commented May 6, 2024 •
edited
Loading

This removes the artefact upload from Ci as it's not super users.

Changes the workflows to use annotations for Cucumber and adds coverage annotations.

Investigating this at the moment, not really sure on the solution.

@nicholasgriffintn nicholasgriffintn requested review from a team as code owners May 6, 2024 15:47
@nicholasgriffintn nicholasgriffintn changed the title chore: swap out artefacts for annotions chore: swap out artefacts for annotations May 6, 2024
@codeclimate CodeClimate
Copy link

codeclimate bot commented May 6, 2024 •
edited
Loading

Code Climate has analyzed commit b0d4bbe and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (80% is the threshold).

This pull request will bring the total coverage in the repository to 98.3% (0.0% change).

View more on Code Climate.

@nicholasgriffintn nicholasgriffintn marked this pull request as draft May 6, 2024 16:10
@nicholasgriffintn nicholasgriffintn changed the title chore: swap out artefacts for annotations chore: enhance ci reports May 6, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented May 6, 2024 •
edited
Loading

LCOV of commit b0d4bbe during Comment PR Coverage #13

Summary coverage rate:
  lines......: 98.4% (842 of 856 lines)
  functions..: 100.0% (34 of 34 functions)
  branches...: 94.1% (159 of 169 branches)

Files changed coverage rate: n/a

@nicholasgriffintn nicholasgriffintn marked this pull request as ready for review May 6, 2024 16:55
@nicholasgriffintn nicholasgriffintn merged commit eded17a into main May 6, 2024
9 checks passed
@nicholasgriffintn nicholasgriffintn deleted the ci-annotations branch May 6, 2024 16:55
@github-actions github-actions bot locked and limited conversation to collaborators May 6, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@nicholasgriffintn